-
Notifications
You must be signed in to change notification settings - Fork 12
feat: add GitHub Actions workflows for build and release #16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
nickbeth
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much for submitting this PR, I've actually wanted to setup proper CI for a long time but never got around doing it.
Sorry for the late reply, I've been quite busy in the past months, so I appreciate any form of contribution.
| license = "MIT" | ||
|
|
||
| [profile.release] | ||
| strip = true # Automatically strip symbols from the binary. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's no real reason to strip debugging symbols from the binary, even for release builds. The executable is already very small, and in the past I had to debug a release build of the app, symbols were really useful in that situation.
| with: | ||
| files: artifacts/**/* | ||
|
|
||
| winget: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wouldn't want this job yet in the repo, I'd suggest to drop it for now so that we can merge the rest of the CI and then once winget is in place, we can re-add it later on.
| - name: Release Artifact Upload | ||
| uses: softprops/action-gh-release@v2 | ||
| with: | ||
| files: artifacts/**/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure how this action works, but is there any way to strip the -release part from the executable name? I'd like the release artifact to just be called wsl-usb-manager.exe.
Summary
This update introduces GitHub Actions workflows to streamline the build and release processes for the
wsl-usb-managerproject.Changes
Added GitHub Actions Workflows:
.github/workflows/build.yaml):.github/workflows/release.yaml):Optimized Cargo Build Configuration:
Cargo.tomlunder[profile.release]:Impact
These changes automate and enhance the project's CI/CD pipeline, reducing manual intervention during builds and releases while providing improved build efficiency and binary optimization for production releases.